docs(#6456): align SKILL.md bot-login refs with GraphQL-format constants by fullsend-ai-coder[bot] · Pull Request #6457 · fullsend-ai/fullsend

fullsend-ai-coder · 2026-08-21T19:28:51Z

Summary

Remove [bot] suffix from three fullsend-ai-review[bot] references in skills/nextwork/SKILL.md (lines 55, 88, 253)
Aligns skill documentation with the REVIEW_BOT_LOGIN constant in nextwork.py, which was updated to use GraphQL-format logins (without [bot] suffix) in PR fix(#6205): strip [bot] suffix from bot-login constants in nextwork.py #6206

Context

PR #6206 fixed bot-login constants to use GraphQL-format logins, but the review agent's finding about stale SKILL.md references was not addressed before merge. This PR completes that cleanup.

Testing

Verified grep -c '\[bot\]' skills/nextwork/SKILL.md returns 0
Verified all three updated references now match the code constant fullsend-ai-review
Markdown link checker (make lint-md-links) passed with 0 errors

Closes #6456

Post-script verification

Branch is not main/master (agent/6456-fix-skill-bot-login-refs)
Secret scan passed (gitleaks — eeeab1a01ddfef5cdaf16424dcd21878cb557eaf..HEAD)
PR body secret scan passed (gitleaks — no-git)

PR #6206 updated REVIEW_BOT_LOGIN in nextwork.py to use the GraphQL-format login `fullsend-ai-review` (without `[bot]` suffix). Three references in skills/nextwork/SKILL.md still used the old REST-format `fullsend-ai-review[bot]`, creating documentation drift. Remove the `[bot]` suffix from lines 55 (--resolve-threads), 88 (waiting_fix status), and 253 (bot_only field) to match the code constants. Note: pre-commit could not run (sandbox network policy blocked hook environment setup). Markdown link checker passed. Closes #6456

fullsend-ai-review · 2026-08-21T19:30:49Z

🤖 Finished Review · ✅ Success · Started 7:30 PM UTC · Completed 7:40 PM UTC

Commit: 8961ae0 · View workflow run →

codecov · 2026-08-21T19:32:28Z

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

fullsend-ai-review · 2026-08-21T19:40:33Z

Review

Findings

Medium

[protected-path] skills/nextwork/SKILL.md — This PR modifies a file under the protected skills/ path. The change is linked to issue Update SKILL.md bot-login references to match GraphQL-format constants from PR #6206 #6456 and the PR description explains the rationale (aligning documentation with GraphQL-format code constants from PR fix(#6205): strip [bot] suffix from bot-login constants in nextwork.py #6206). Human approval is always required for protected-path changes, regardless of context.

fullsend-ai-retro · 2026-08-27T08:27:11Z

🤖 Finished Retro · ✅ Success · Started 8:27 AM UTC · Completed 8:39 AM UTC

Commit: 8961ae0 · View workflow run →

Runtime: claude · Model: opus → claude-opus-4-6 · Cost: $3.62

fullsend-ai-retro · 2026-08-27T08:39:38Z

Retro: PR #6457 — align SKILL.md bot-login refs with GraphQL-format constants

Timeline

This PR is the result of a full end-to-end automation loop that worked cleanly:

PR fix(#6205): strip [bot] suffix from bot-login constants in nextwork.py #6206 (merged Aug 21) fixed bot-login constants in nextwork.py from REST format (fullsend-ai-review[bot]) to GraphQL format (fullsend-ai-review). The review agent (run 31738319224) correctly flagged a low-severity stale-identifier-reference noting that skills/nextwork/SKILL.md still used the old [bot] suffix in three places. The human reviewer approved without addressing it.
Retro agent filed issue #6456 ~20 minutes after PR fix(#6205): strip [bot] suffix from bot-login constants in nextwork.py #6206 merged, with exact file paths, line numbers, and validation criteria.
Triage agent (run 32517670171) confirmed the issue in ~4 minutes.
Code agent (run 32518137236) fixed all 3 references correctly on the first attempt in ~6.5 minutes. Pre-commit hooks failed in sandbox due to network policy, but the harness's authoritative post-script pre-commit run succeeded.
Review agent (run 32518680566) flagged only a medium-severity protected-path finding for skills/ — appropriate for the directory.
Human reviewer (rh-hemartin) approved and merged on Aug 27, ~6 days after PR creation.

What went well

Zero rework: Code agent got the fix right on the first attempt. No review-requested changes, no fix agent runs.
Retro agent value: Identified a real finding that the human reviewer on PR fix(#6205): strip [bot] suffix from bot-login constants in nextwork.py #6206 did not address, filed a well-structured issue, and the system self-healed.
Fast automation: Issue → triaged → coded → PR in ~12 minutes.
Review accuracy: The review agent's stale-identifier-reference finding on PR fix(#6205): strip [bot] suffix from bot-login constants in nextwork.py #6206 was accurate and well-described.

Existing issues with new evidence

Protected-path severity for bot-authored doc-only PRs (Review agent: downgrade protected-path severity for human-authored PRs #1551, Review agent: downgrade protected-path severity for bot-authored dependency update PRs #2588, Review agent should downgrade protected-path for platform bot managed-file updates #3164, Suppress protected-path finding when all changed files are in protected paths #1392): This PR is another data point — a trivial 3-line doc fix from a trusted bot required manual review solely due to the protected-path flag on skills/. The requires-manual-review label contributed to the 6-day merge delay.
Surface stale agent PRs awaiting human review (Surface agent-approved PRs awaiting human review after N days #966, Priority-aware escalation for agent-approved PRs awaiting human review #1974): The 6-day delay between PR creation (Aug 21) and merge (Aug 27) for a trivial, CI-passing doc fix suggests human review is a bottleneck for low-priority bot PRs.
Code agent identifier cross-referencing (Code agent should verify all inbound cross-references when renaming markdown headings #2913, AGENTS.md: add reverse-direction docs-to-code verification guidance #6291): See proposal below for a specific improvement targeting code→docs direction.

Agents repo

All agents resolved from fullsend-ai/agents at commit 12f5d88b5d8cc4ab12ba1c6332c6c512deaa0b0e.

Proposals filed

Code agent: sweep same-module docs for stale identifier references when changing constants (in fullsend-ai/agents)

fullsend-ai-coder Bot requested a review from a team as a code owner August 21, 2026 19:28

fullsend-ai-coder Bot added the ready-for-review Agent PR ready for human review label Aug 21, 2026

fullsend-ai-coder Bot temporarily deployed to dev August 21, 2026 19:29 Inactive

fullsend-ai-review Bot added the requires-manual-review Review requires human judgment label Aug 21, 2026

rh-hemartin approved these changes Aug 27, 2026

View reviewed changes

rh-hemartin added this pull request to the merge queue Aug 27, 2026

Merged via the queue into main with commit 8266cbb Aug 27, 2026
54 of 55 checks passed

rh-hemartin deleted the agent/6456-fix-skill-bot-login-refs branch August 27, 2026 08:25

fullsend-ai-retro Bot mentioned this pull request Aug 27, 2026

Code agent: sweep same-module docs for stale identifier references when changing constants fullsend-ai/agents#1065

Open

Navigation Menu

Uh oh!

docs(#6456): align SKILL.md bot-login refs with GraphQL-format constants - #6457

docs(#6456): align SKILL.md bot-login refs with GraphQL-format constants#6457
rh-hemartin merged 1 commit into
mainfrom
agent/6456-fix-skill-bot-login-refs

fullsend-ai-coder Bot commented Aug 21, 2026

Uh oh!

fullsend-ai-review Bot commented Aug 21, 2026 •

edited

Loading

Uh oh!

codecov Bot commented Aug 21, 2026

Uh oh!

fullsend-ai-review Bot commented Aug 21, 2026

Uh oh!

Uh oh!

fullsend-ai-retro Bot commented Aug 27, 2026 •

edited

Loading

Uh oh!

fullsend-ai-retro Bot commented Aug 27, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Uh oh!

Conversation

fullsend-ai-coder Bot commented Aug 21, 2026

Summary

Context

Testing

Post-script verification

Uh oh!

fullsend-ai-review Bot commented Aug 21, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

codecov Bot commented Aug 21, 2026

Codecov Report

Uh oh!

fullsend-ai-review Bot commented Aug 21, 2026

Review

Findings

Medium

Uh oh!

Uh oh!

fullsend-ai-retro Bot commented Aug 27, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

fullsend-ai-retro Bot commented Aug 27, 2026

Retro: PR #6457 — align SKILL.md bot-login refs with GraphQL-format constants

Timeline

What went well

Existing issues with new evidence

Agents repo

Proposals filed

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

fullsend-ai-review Bot commented Aug 21, 2026 •

edited

Loading

fullsend-ai-retro Bot commented Aug 27, 2026 •

edited

Loading